Skip to content

Remove deprecated Python settings - #625

Merged
Samruddhi Khandale (samruddhikhandale) merged 5 commits into
devcontainers:mainfrom
luabud:removedeprecatedsettings
Oct 2, 2023
Merged

Remove deprecated Python settings#625
Samruddhi Khandale (samruddhikhandale) merged 5 commits into
devcontainers:mainfrom
luabud:removedeprecatedsettings

Conversation

@luabud

@luabud Luciana Abud (luabud) commented Jul 21, 2023

Copy link
Copy Markdown
Contributor

Hi there!

The python.linting and python.formatting settings have been deprecated as in the latest Python extension pre-release version, as we're migrating our tooling support to extensions (see here for more details).

We figured we'd submit PRs to remove them from devcontainers configs 😊

@luabud
Luciana Abud (luabud) marked this pull request as ready for review July 21, 2023 20:08
@luabud
Luciana Abud (luabud) requested a review from a team as a code owner July 21, 2023 20:08

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for taking the time to contribute this PR, we appreciate it! ✨

Comment thread src/python/devcontainer-feature.json
Comment thread src/python/devcontainer-feature.json
@luabud

Copy link
Copy Markdown
Contributor Author

Took the liberty to remove the linters and formatters from DEFAULT_UTILS, but happy to revert if it not appropriate! 😊

@luabud

Copy link
Copy Markdown
Contributor Author

Samruddhi Khandale (@samruddhikhandale) friendly ping 😊

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the ping, apologies to have missed this!
Left some comments.

Comment thread src/python/devcontainer-feature.json
Comment thread src/python/devcontainer-feature.json
@luabud

Copy link
Copy Markdown
Contributor Author

Last commit reverted 😊

@luabud

Copy link
Copy Markdown
Contributor Author

Samruddhi Khandale (@samruddhikhandale) friendly ping!

@samruddhikhandale

Copy link
Copy Markdown
Member

Luciana Abud (@luabud) I had been waiting for this comment https://github.com/devcontainers/features/pull/625/files#r1286251867

Can you help resolve it, then we could merge this PR. Thanks!

@luabud

Copy link
Copy Markdown
Contributor Author

Samruddhi Khandale (@samruddhikhandale) oh I'm so sorry! I had totally misunderstood that this would be done in a follow up clean up PR (which I now realize was about bumping the major version, instead of minor 🤦‍♀️)

Just bumped the version to it to 1.2.1 :) thanks a lot!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you so much! 🎉

@samruddhikhandale
Samruddhi Khandale (samruddhikhandale) merged commit c3f9105 into devcontainers:main Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants